-
Notifications
You must be signed in to change notification settings - Fork 924
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Artemis Console next gen based on hawtIO 4 #5149
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looked over some of it (not the tests) and noted some small things
artemis-cli/src/main/java/org/apache/activemq/artemis/cli/commands/Upgrade.java
Show resolved
Hide resolved
tests/smoke-tests/src/main/filtered-resources/servers/linuxUpgradeETC/artemis.profile
Outdated
Show resolved
Hide resolved
Couple of things I noticed right off the bat...
|
I dont recall either way about the first issue you listed (didnt spot it though if it was there), but I didnt notice the 2nd or 3rd. Seems like one of us possibly had something cached. I'll give it another build tomorrow and check. I did notice the 4th about the log messages but forgot to post it in my final summary comment. |
@gemmellr, so I restarted the broker, opened the console again, and forced a complete reload and the images appeared properly. That takes care of issues 2, 3, & 5! |
I do see this one too. It is logging from a hawtio class, and seems like it will go away with hawtio 4.1.0 as they switched it to debug: |
I saw the exception.. I couldn't get the web console to run.. was I supposed to do anything to be able to run it? |
The warning log messaged from hawtio discussed before? Or Something else?
Just build + start the broker. When you say you couldnt get it to run, what exactly did you encounter? An error loading the console page, or did you just not try it because it didnt log its URL (which hasnt changed) at startup? Per earlier discussion, it wasnt logging the messages as before, but was still there. |
<dependency> | ||
<groupId>org.apache.activemq</groupId> | ||
<artifactId>activemq-branding</artifactId> | ||
<type>war</type> | ||
</dependency> | ||
<dependency> | ||
<groupId>org.apache.activemq</groupId> | ||
<artifactId>artemis-plugin</artifactId> | ||
<type>war</type> | ||
</dependency> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is as much a comment on the unaltered bit remaining, as the removals.
The binary distribution assembly has a number of entries in its LICENCE file (and files it references) to do with covering the console bits. Those files haven't been changed here, but it seems almost certain they will need to be. That is, artemis-distribution/src/main/resources/licenses/bin/LICENSE
and files under dir artemis-distribution/src/main/resources/licenses/bin/licenses/
.
I added some commits fixing many of the code items I commented on earlier. I also switched it to the current SNAPSHOT since the staging repo was dropped when the vote was cancelled, so the build wouldn't work at all otherwise. Remaining open points are the licence stuff and other things discussed in the comments, e.g startup log messages / warnings. |
which references do you refer to @jbertram ?
|
closing and replaced by #5245 |
This is a PR to allow developers to build a variant of the latest Artemis Release with the new Artemis Console embedded, for testing. It will not currently merge and isnt intended to.
I have staged the 1st Artemis Console release candidate, 1.0.0, and this build will use the staged dependency when built.
I will make this PR a draft and update it for each Artemis Release until it is decided to merge into main.